-
Notifications
You must be signed in to change notification settings - Fork 14.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove duplicate error message on chart connection failure #8476
Conversation
@@ -160,9 +160,6 @@ <h4 class="panel-title"> | |||
url = "{{ url_for('airflow.chart_data') }}" + location.search; | |||
$.getJSON(url, function(payload) { | |||
$('#loading').hide(); | |||
if (payload.error !== undefined) { | |||
$('#chart_body').html('<div class="alert alert-danger">' + payload.error + '</div>'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The error is still shown by the error function on L181
This caused the error message to be shown twice - once in the chart body (which is faded out) and again at the "top-level" error
91b6c46
to
f8a3f91
Compare
Codecov Report
@@ Coverage Diff @@
## v1-10-test #8476 +/- ##
==============================================
- Coverage 81.86% 81.30% -0.56%
==============================================
Files 542 542
Lines 37221 37223 +2
==============================================
- Hits 30471 30265 -206
- Misses 6750 6958 +208
Continue to review full report at Codecov.
|
This caused the error message to be shown twice - once in the chart body (which is faded out) and again at the "top-level" error (cherry picked from commit ecf0494)
This caused the error message to be shown twice - once in the chart body (which is faded out) and again at the "top-level" error (cherry picked from commit ecf0494) (cherry picked from commit b779c59c1303faa14ac32204fa7afc169441531b)
This caused the error message to be shown twice - once in the chart body (which is faded out) and again at the "top-level" error (cherry picked from commit ecf0494) (cherry picked from commit b779c59c1303faa14ac32204fa7afc169441531b)
This caused the error message to be shown twice - once in the chart body (which is faded out) and again at the "top-level" error
This caused the error message to be shown twice - once in the chart body
(which is faded out) and again at the "top-level" error
[AIRFLOW-XXXX]
for document-only changesIn case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.
Read the Pull Request Guidelines for more information.